Skip to content

Document GraphQL API for GA release #264

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
May 7, 2025

Conversation

rsill-neo4j
Copy link
Contributor

No description provided.

@rsill-neo4j
Copy link
Contributor Author

is this only going to be available in 7.x?
i'm asking because we had the beta docs in 6.x as well as 7.x @LackOfMorals

@Liam-Doodson
Copy link
Contributor

is this only going to be available in 7.x? i'm asking because we had the beta docs in 6.x as well as 7.x @LackOfMorals

It's a new product that will only support v7 of the library. I think it should probably only be included in the v7 version of the docs going forwards as users looking at v6 or older would need to upgrade to v7 before they can make use of the feature anyway

Copy link
Contributor

@Liam-Doodson Liam-Doodson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's slightly jarring having essentially a getting started guide for both the console and cli mixed together. Would it be feasible to refactor this into two separate guides?

This would also make it easier to add docs on how to enable to CLI feature flag. We also might want to add "Getting Started Using the Aura API" docs in the future and they'd fit better the separate getting started guide format imo

I know there's not too long to get this done though as the GA just around the corner so it might be something for a follow-up PR



== Aura CLI

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just seen the docs on using the CLI, maybe this should just link to those?

rsill-neo4j and others added 3 commits April 29, 2025 08:29
Co-authored-by: Liam-Doodson <114480811+Liam-Doodson@users.noreply.github.com>
works for me

Co-authored-by: Liam-Doodson <114480811+Liam-Doodson@users.noreply.github.com>
Copy link
Collaborator

@LackOfMorals LackOfMorals left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me

Copy link
Contributor

@Liam-Doodson Liam-Doodson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just added a few comments for consistency of the commands but other than those this LGTM

rsill-neo4j and others added 6 commits April 30, 2025 09:05
Co-authored-by: Liam-Doodson <114480811+Liam-Doodson@users.noreply.github.com>
Co-authored-by: Liam-Doodson <114480811+Liam-Doodson@users.noreply.github.com>
@rsill-neo4j rsill-neo4j merged commit 5551d66 into 7.x May 7, 2025
4 checks passed
@neo4j-docops-agent
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants